Skip to content

BDMS 221: updates to transfer scripts#242

Merged
jacob-a-brown merged 23 commits into
bdms-221from
bdms-221-jab-transfer-updates
Nov 18, 2025
Merged

BDMS 221: updates to transfer scripts#242
jacob-a-brown merged 23 commits into
bdms-221from
bdms-221-jab-transfer-updates

Conversation

@jacob-a-brown

Copy link
Copy Markdown
Contributor

this PR is to update transfer scripts for the changes made to models/schemas/API as we implement feature tests. This is dependent on PR #240 being merged.

Comment thread db/thing.py

@jirhiker jirhiker left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking pretty good

Comment thread schemas/thing.py Outdated
Comment thread db/thing.py
Comment thread transfers/group_transfer.py Outdated

@TylerAdamMartinez TylerAdamMartinez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jirhiker jirhiker left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 non blocking / technical debt/style comments

Comment thread transfers/util.py Outdated
Comment thread transfers/well_transfer.py Outdated
@jacob-a-brown jacob-a-brown merged commit 784cb30 into bdms-221 Nov 18, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants